Skip to content

Conversation

seunghun1ee
Copy link
Member

@seunghun1ee seunghun1ee commented Jul 3, 2025

The revert at stackhpc/terraform-kayobe-multinode#92 made Epoxy and Caracal ci-multinode to fail because SKC assumes that OpenBao is used as CA but multinode.sh at terraform-kayobe-multinode assumes vault.

This merge allows users to choose which secret store to use as CA.

Related PR: stackhpc/terraform-kayobe-multinode#94

@seunghun1ee
Copy link
Member Author

/cherry-pick stackhpc/2024.1

MaxBed4d
MaxBed4d previously approved these changes Jul 4, 2025
Copy link
Contributor

@MaxBed4d MaxBed4d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@Alex-Welsh Alex-Welsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're planning on making the vault* and openbao* playbooks generic, and use a separate variable to choose between them

See discussion here for details: stackhpc/terraform-kayobe-multinode#93 (comment)

@Alex-Welsh
Copy link
Member

I'm putting this PR back to draft while we work on a new solution and Seunghun is away

@seunghun1ee seunghun1ee force-pushed the ci-multinode-ca-choice branch from fe2c6c2 to f47a2ac Compare July 18, 2025 14:03
@seunghun1ee seunghun1ee changed the title Ci multinode ca choice Merge Openbao and Hashicorp Vault playbooks Jul 18, 2025
@bbezak bbezak disabled auto-merge July 18, 2025 14:12
@bbezak bbezak merged commit ff523e8 into stackhpc/2025.1 Jul 18, 2025
14 of 20 checks passed
@bbezak bbezak deleted the ci-multinode-ca-choice branch July 18, 2025 14:12
Copy link

Cherry-pick failed with Merge error ff523e8500d7ec2e3d1f6f7e53e97b326b600136 into temp-cherry-pick-5af7fb-stackhpc/2024.1

@seunghun1ee seunghun1ee added ansible Ansible playbooks Epoxy and removed workflows Workflow files have been modified labels Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants